gtk: Fix XIGrabKeycode usage
authorBastien Nocera <hadess@hadess.net>
Thu, 27 Oct 2011 10:09:26 +0000 (11:09 +0100)
committerBastien Nocera <hadess@hadess.net>
Fri, 28 Oct 2011 11:03:22 +0000 (12:03 +0100)
A modifier of "0" is still a modifier and must be submitted normally.
A num_modifiers of 0 is essentially a noop in the server.

As mentioned by Peter Hutterer <peter.hutterer@who-t.net>

https://bugzilla.gnome.org/show_bug.cgi?id=662839

gtk/gtkdnd.c

index 1b8baa45da1c15efe412a17710ee00bbc2977149..e1164252a532e18918d848c5a1d2c68cf0d043c5 100644 (file)
@@ -513,13 +513,8 @@ grab_dnd_keys (GtkWidget *widget,
           evmask.mask_len = sizeof (mask);
           evmask.mask = mask;
 
-          if (grab_keys[i].modifiers != 0)
-            {
-              num_mods = 1;
-              mods.modifiers = grab_keys[i].modifiers;
-            }
-          else
-            num_mods = 0;
+          num_mods = 1;
+          mods.modifiers = grab_keys[i].modifiers;
 
           XIGrabKeycode (GDK_WINDOW_XDISPLAY (window),
                          deviceid,
@@ -591,13 +586,8 @@ ungrab_dnd_keys (GtkWidget *widget,
 #ifdef XINPUT_2
       if (using_xi2)
         {
-          if (grab_keys[i].modifiers != 0)
-            {
-              num_mods = 1;
-              mods.modifiers = grab_keys[i].modifiers;
-            }
-          else
-            num_mods = 0;
+          num_mods = 1;
+          mods.modifiers = grab_keys[i].modifiers;
 
           XIUngrabKeycode (GDK_WINDOW_XDISPLAY (window),
                            deviceid,